Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Begin to establish 1:1 mapping between media_id (xklb-metadata.db) <-> Calibre-Web's book_id (metadata.db) #255

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

deldesir
Copy link
Collaborator

@deldesir deldesir commented Sep 12, 2024

This pull request aims to initiate the process of creating a one-to-one mapping between the media_id in xklb-metadata.db and the book_id in Calibre-Web's metadata.db. This enhancement is crucial for maintaining synchronization across the databases, enabling accurate search results based on media entries' captions and their corresponding book records.

Key changes introduced in this pull request include:

  • Mapping Functions: Initial implementation of functions to handle the creation and management of mappings between media_id and book_id. These functions ensure that each media entry in xklb-metadata.db is accurately linked to its corresponding book record in metadata.db.
  • Database Queries: Updates to database queries to incorporate the new mapping logic in the search feature, ensuring that operations do not lead to false positives.

This update builds on the foundation laid in previous pull requests (#244 and #256) and aims to enhance the overall structure and maintainability of the codebase.

  • Tested on Ubuntu 24.04

@deldesir deldesir added the enhancement New feature or request label Sep 12, 2024
@deldesir deldesir requested a review from holta September 12, 2024 16:05
@deldesir deldesir self-assigned this Sep 12, 2024
@deldesir deldesir marked this pull request as draft September 25, 2024 02:12
@avni
Copy link
Member

avni commented Oct 14, 2024

@deldesir indicates that this PR will need to be changed and use iiab-glue.db because of the DB refactoring we have been discussing so that we map books and media and also minimize impact due to upstream impact. 🚀

@holta holta changed the title Map media_id with relevant book_id Begin to establish 1:1 mapping between media_id (xklb-metadata.db) <-> Calibre-Web's book_id (metadata.db) Oct 14, 2024
@deldesir deldesir marked this pull request as ready for review October 14, 2024 20:39
@deldesir
Copy link
Collaborator Author

deldesir commented Oct 17, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants